x11/gl: Do not call glEnable/glDisable with GL_TEXTURE_2D
authorEmmanuele Bassi <ebassi@gnome.org>
Mon, 23 Mar 2015 22:03:00 +0000 (22:03 +0000)
committerEmmanuele Bassi <ebassi@gnome.org>
Mon, 23 Mar 2015 22:03:00 +0000 (22:03 +0000)
These are fixed pipeline functions, and Mesa will warn (if debugging
messages are enabled). At best, they are poinless calls.

gdk/x11/gdkglcontext-x11.c

index e2d70e9b73d8890f582f626aecb8b279db66ded0..82f50710d7279298ec5f9b4fa1557589c5e8565b 100644 (file)
@@ -461,7 +461,10 @@ gdk_x11_gl_context_texture_from_surface (GdkGLContext *paint_context,
 
   glGenTextures (1, &texture_id);
   glBindTexture (target, texture_id);
-  glEnable (target);
+
+  /* glEnable(GL_TEXTURE_2D) is deprecated */
+  if (target != GL_TEXTURE_2D)
+    glEnable (target);
 
   glTexParameteri (target, GL_TEXTURE_WRAP_S, GL_CLAMP_TO_EDGE);
   glTexParameteri (target, GL_TEXTURE_WRAP_T, GL_CLAMP_TO_EDGE);
@@ -526,7 +529,9 @@ gdk_x11_gl_context_texture_from_surface (GdkGLContext *paint_context,
   glXReleaseTexImageEXT (glx_pixmap->display, glx_pixmap->drawable,
                         GLX_FRONT_LEFT_EXT);
 
-  glDisable (target);
+  if (target != GL_TEXTURE_2D)
+    glDisable (target);
+
   glDeleteTextures (1, &texture_id);
 
   glx_pixmap_destroy(glx_pixmap);